Skip to content

[Minesweeper - (Soon to be Flower Field)] draft approaches #3663

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
wants to merge 2 commits into from
Closed

[Minesweeper - (Soon to be Flower Field)] draft approaches #3663

wants to merge 2 commits into from

Conversation

colinleach
Copy link
Contributor

Just the introduction.md in this case. I couldn't identify such clearly different approaches that would justify separate sections.

I'll read this through again on Saturday, so there may be some minor corrections added.

@colinleach
Copy link
Contributor Author

I made a few tweaks but couldn't think of anything major,

Over to you for comments. Meanwhile, I'll see how many more of these I can get drafted before this burst of enthusiasm drains away.

ABcDexter
ABcDexter reviewed Oct 26, 2024
View reviewed changes
Copy link

@ABcDexter ABcDexter left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These look good :)

@BethanyG
Copy link
Member

@ABcDexter -- Reviews in this repo should only be done by the maintainer team. Thanks!

@exercism exercism locked as off-topic and limited conversation to collaborators Oct 26, 2024
@BethanyG
Copy link
Member

Given this change, going to take this back to draft. After the deprecation/relaunch, we can then explain and revise this.

@BethanyG BethanyG marked this pull request as draft June 25, 2025 17:24
@BethanyG BethanyG changed the title [Minesweeper] draft approaches [Minesweeper - Soon to be Flower Field] draft approaches Jun 25, 2025
@BethanyG BethanyG changed the title [Minesweeper - Soon to be Flower Field] draft approaches [Minesweeper - (Soon to be Flower Field)] draft approaches Jun 25, 2025
@exercism exercism unlocked this conversation Jun 25, 2025
@BethanyG
Copy link
Member

@colinleach So I think the thing to do here is to take what we have and copy it into the flower-field exercise. (i.e. a new PR).

We can then change the mine and minesweeper references and re-review. We can then close this PR in favor of the new flower field one.

Happy to have you do that, or I can. Either works. 😄 Just let me know!

@colinleach
Copy link
Contributor Author

Sounds simple enough that I'll do it later today.

@colinleach
Copy link
Contributor Author

It's funny to re-read this 15 months later.

Meanwhile, I'll see how many more of these I can get drafted before this burst of enthusiasm drains away

That draining didn't take long, for either of us!

@colinleach
Copy link
Contributor Author

Replaced by #3935

@colinleach colinleach closed this Jun 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ABcDexter ABcDexter ABcDexter left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@colinleach @BethanyG @ABcDexter